-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add rust-analyzer proc-macro server project #24
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks pretty good already! Let me know once you un-draft it when you want to merge it.
I want to sketch up a rough plan / roadmap first. Will either do so today or tomorrow |
Ok! I think that what you have in "Expected result" is in fact a roadmap (although it would be great if you can further extend it, of course!), and the expected result should just be something like "There exists a proc-macro server that is more efficient and supports more use-cases. Ideally, it should be integrated within RA/RustRover.", or something like that. |
e5de328
to
0d2a1ce
Compare
Okay I replaced the expected results part and created a tracking issue in the r-a repo that outlines more finer details rust-lang/rust-analyzer#19205 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, this sounds like a great fit for GSoC indeed! Left one nit.
0d2a1ce
to
2c582ef
Compare
Pushed a link anchor fix, I missed it previously. Thank you! |
No description provided.